From: Sergio Costas Rodriguez <sergio.costas@canonical.com>
Date: Thu, 16 Apr 2026 11:41:47 +0000 (+0200)
Subject: dash: Don't notify two elements for each icon
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~10^2^2~36
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=3df8e4dd0e14ecc4221ef5f05eebeac1c9949a3d;p=gnome-shell.git

dash: Don't notify two elements for each icon

When the user uses a screen reader and navigates through the
dash, each element is notified as two: one panel with no label,
and then a button with the application name as the label.

This is because each icon has the BUTTON role, but is inside a
container with the PANEL role, so each time the focus changes
from an icon to the next, there is a change in the container,
so the screen reader notifies this. But that container is
useless for the screen reader. The container always read its
label because the label_actor property is set.

This patch fixes this.

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/4182>
---

diff --git a/js/ui/dash.js b/js/ui/dash.js
index 8dc16dc6a..9ebd348ee 100644
--- a/js/ui/dash.js
+++ b/js/ui/dash.js
@@ -65,7 +65,6 @@ class DashItemContainer extends St.Widget {
         this.label.hide();
         Main.layoutManager.addChrome(this.label);
         this.label.connectObject('destroy', () => (this.label = null), this);
-        this.label_actor = this.label;
 
         this.child = null;
         this.animatingOut = false;